Skip to content

Conversation

@cgarciae
Copy link
Collaborator

What does this PR do?

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@cgarciae cgarciae force-pushed the hijax-variable branch 6 times, most recently from 803cd5d to 2e96cee Compare October 31, 2025 16:48
@cgarciae cgarciae force-pushed the deprecate-value-2 branch 2 times, most recently from 0d0ecb0 to b9f2302 Compare November 11, 2025 18:41
@cgarciae cgarciae changed the base branch from hijax-variable to main November 11, 2025 18:45
)


def str_flag(name: str, *, default: str, help: str) -> FlagHolder[str]:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this part of the PR? I didn't see any flags using it

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah it was outdated. Updated the branch.

@cgarciae cgarciae force-pushed the deprecate-value-2 branch 3 times, most recently from 880d881 to 12bfb60 Compare November 12, 2025 01:21
@cgarciae cgarciae force-pushed the deprecate-value-2 branch 8 times, most recently from 73bf11e to dbef8bf Compare November 15, 2025 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants